Adapt icontheme tests

With our new, improved icon theme spec compliance, some of the
tests need updates to their expected outcomes.

GtkIconInfo: Small struct packing improvement

This makes the GtkIconInfo struct a bit smaller.

GtkIconInfo: Drop an unused field

The GtkIconInfo threshold field was never used, so lets not
carry it around.

GtkIconTheme: Respect limits in theme dirs

Previously, we were taking thresholds and min/max sizes into
account when choosing the best theme dir, but when it came
to loading the icon, we always scaled icons from scalable
directories all the way, ignoring the min/max size limits.

This commit changes things around so that we now load icons
in Threshold directories at their nominal size, and scale
icons in Scalable directories only up to the specified limits.

To override this and keep the previous behaviour of scaling
all the way to the desired size, use the GTK_ICON_THEME_FORCE_SIZE
flag.

gdkwindow: Provide a default implementation of process_updates_recurse

As a quick code cleanup.

gdkwindow-broadway: Move the dirty flag setting to end_paint

This is where it belongs -- process_updates_recurse is the wrong place
for it.

gdkwindow: Move a variable closer to where it's used

To make the code clearer.

gdkwindow: Remove an unused variable

GtkSwitch: Implement hover for GtkSwitch

This makes switches more similar to other activatable widgets
like buttons or scales.

gdkwidow: Make queue_antiexpose optional

gdkwindow-win32: Remove queue_antiexpose implementation

Since the Win32 code never actually called InvalidateRgn or used the
Win32 update area at all, that meant the only thing that could possibly
invalidate the window was the Win32 window manager as part of scrolling
or resizing, which would also send it a WM_PAINT message.

But the WM_PAINT handling called BeginPaint / EndPaint, which clears the
update area completely! We also draw out-of-band, not directly when
handling WM_PAINT, so there's no way that the update area inside the
Win32 WM would match our local one.

There is no possible way that this queue_antiexpose implementation could
do anything. Remove it.

gdkevents-win32: Remove dead code

gdkwindow: Don't bother with a return parameter for queue_antiexpose

Standard refcounting works perfectly well. Don't give us the opportunity
for more memory leaks.

gdkwindow-broadway: Squash memory leak

We don't need to hang on to the update_area.

gdkwindow-broadway: Make queue_antiexpose implementation private

gdkwindow-wayland: Attach new buffers and submit damage in end_paint

process_updates_recurse is simply the wrong place for it to be.

gdkwindow-wayland: Rename listener -> frame_listener

Don't pollute the static namespace here.

docs: add missing apostrophe

gdkwindow: Correct a signature

We removed the parameter from callers and from the implementation, but
not from the signature up top. I didn't notice because the branch I was
working on removed the signature entirely.

gdkwindow: Remove an extra region copy

This code is only called with the current paint region as its argument.
Instead of having to copy it and do a no-op intersect against itself,
just use the current paint directly.

gdkwindow: Clean up code

Introduce a temporary to make the code a bit more clear under these cases.

gdkwindow: Clean up formatting

Tabs to spaces, add some whitespace.

gdkwindow: Add some extra documentation to gdk_cairo_create

Mention the need to wrap the function inside
gdk_window_begin_paint_region and gdk_window_end_paint.

widgetpath: Deprecate regions

This is a foolowup to d80bf0790d0903ba2825f306b6f7435529f922e2

gtkwidget: Add a warning about set_double_buffered

As requested on the mailing list.

testwindows: Remove another reference to set_double_buffered

gtktrayicon-x11: Stop using set_double_buffered

There's nothing better we can do for this case, now that we always
redirect drawing to a temporary pixmap. Maybe since this is already
X11-specific code, we should just do everything with Xlib directly.

Adwaita: Fix highlight on check buttons

The .button style class does not apply to check and radio
buttons, and .check only applies to menuitems.

gdkwindow: Squash a leak in the subsurface case

cairo_surface_create_for_rectangle takes a ref on the parent surface,
so we need to drop ours.

Rename get_window_surface to ref_window_surface to make the code more
clear and to stop this error from happening again.

gdkwindow: Don't create a subsurface if we don't need to

In the case we have a native window, we don't need to crop it to a
non-native subrectangle, as abs_x / abs_y / width / height are all 0.

gdkwindow: Remove the ability to call begin_paint_region more than once

Previously, each begin_paint_region added to a stack of current paints,
and when end_paint was called, the paint was popped off of the stack and
the surface was composited into the parent paint surface.

However, the code was broken in the case of a backend like Wayland which
didn't keep track of nested calls and simply wiped and returned the
native impl backing surface every time.

Since this feature is flat out unused by GTK+ and we don't want to
really support tricksy things like these for other clients, just remove
the feature. If somebody does call begin_paint_region more than once,
warn and return without doing anything.

gdkwindow: Remove the internal cairo_surface used for out-of-band painting

Traditionally, the way painting was done in GTK+ was with the
"expose-event" handler, where you'd use GDK methods to do drawing on
your surface. In GTK+ 2.24, we added cairo support with gdk_cairo_create,
so you could paint your graphics with cairo.

Since then, we've added client-side windows, double buffering, the paint
clock, and various other enhancements, and the modern way to do drawing
is to connect to the "draw" signal on GtkWidget, which hands you a
cairo_t. To do double-buffering, the cairo_t we hand you is actually on
a secret surface, not the actual backing store of the window, and when
the draw handler completes we blit it into the main backing store
atomically.

The code to do this is with the APIs gdk_window_begin_paint_region,
which creates the temporary surface, and gdk_window_end_paint which
blits it back into the backing store. GTK+'s implementation of the
"draw" signal uses these APIs.

We've always sort-of supported people calling gdk_cairo_create
"outside" of a begin_paint / end_paint like old times, but then you're
not getting the benefit of double-buffering, and it's harder for GDK to
optimize.

Additionally, newer backends like Mir and Wayland can't actually support
this model, since they're based on double-buffering and swapping buffers
at various points in time. If we hand you a random cairo_t, we have no
idea when is a good time to swap.

Remove support for this.

This is technically a GDK API break: a warning is added in cases where
gdk_cairo_create is called outside of a paint cycle, and the returned
surface is a dummy that won't ever be composited back onto the main
surface. Testing with complex applications like Ardour didn't produce
any warnings.

gtkwidget: Widgets are now always double-buffered

gtk_widget_set_double_buffered is now deprecated, and we don't support
non-double-buffered widgets. This means that under normal circumstances,
paints are never outside of a begin_paint / end_paint sequence, which
natively-double-buffered backends like Wayland can't possibly support.

gdkwindow: Deprecate gdk_window_flush

The function does nothing.

GtkIconTheme: Drop an unused function

GtkIconTheme: Small cleanup

Make the default symbolic colors a little more readable in the code.

GtkIconTheme: Trivial cleanup

Drop the _-prefix from a static function.
We've agreed to no stop using those prefixes.

Avoid a compiler warning

The name_offset variable is indeed unused.

Big spinner

GtkIconTheme: Don't create empty IconThemeDir structs

Starting gtk3-demo currently creates ~1200 of these, most
of them for nonexisting directories. Lets try to avoid that.

GtkIconCache: Add api to find if directories are empty

This will help in not creating structs for tons of empty
directories.

GtkStatusbar: Add some space between the frame and the label

Looks better this way.

Adwaita: check/radio selector simplification

widget factory: Use a property action for the toolbar toggle

This is slightly more elegant than doing it manually.

widget factory: Add a statusbar

A widget that is not represented in the widget factory is likely
to not get proper attention from theme authors.

docs: Drop image references for stock items

Most of those images are no longer in our tree, so just
say which ones have icons and rtl variants, instead of
showing it.

wayland: Prevent stale paints and weird artifacts when using Weston

Weston releases buffers almost immediately after they're done, which
means that GTK+ doesn't use a temporary surface and instead paints
directly onto the SHM backing store that Weston will use.

Normally, after painting to the temporary surface, GTK+ *replaces*
the existing backing surface with CAIRO_OPERATOR_SOURCE. However,
if we immediately paint to the backing surface, it might have junk
from the last paint in it. So clear out the backing surface whenever
somebody calls begin_paint_region().

Maybe we should just always use the temporary surface like the X11
codepath, since that prevents us from having to do weird things like
this, but oh well.

GtkIconTheme: Keep gtk_icon_theme_add_resource_path private for now

We need to discuss some details of the semantics before
we can make this public.

Revert "Document gtk_icon_theme_add_resource_path"

This reverts commit 988b6b483dc9c21d050161d26652bb1006b9c686.

Revert "Deprecate gtk_icon_theme_add_builtin_icon"

This reverts commit 923eeda2d3cae13d02b0a9d2e5774b12eed999e0.

Deprecate gtk_icon_theme_add_builtin_icon

Document gtk_icon_theme_add_resource_path

Adapt the icontheme tests

With the new way of dealing with the legacy stock icons,
the builtin test needed some small updates.

GtkIconTheme: Include legacy stock icons from resources

Add /org/gtk/libgtk/icons as a resource path, and ensure
that we always parse an index.theme file for hicolor which
makes the builtin icons available as part of the hicolor theme.

GtkIconTheme: Allow loading icons from resources

We add a new API, gtk_icon_theme_add_resource_path, which
can be used to add resource path as a base location for
icon theme content, similar to gtk_icon_theme_append_search_path.

GtkIconTheme: Drop the all_icons hash table

It is not used anywhere anymore.

GtkIconTheme: Stop using the all_icons hash table

This hash table was used in gtk_icon_theme_has_icon; we can
implement that function just as well without it and save some
memory.

icontheme: Add some more tests

This test checks that gtk_icon_theme_list_icons and
gtk_icon_theme_has_icon work as expected.

Formatting fixes

GtkIconTheme: Drop the code for parsing .icon files

Modern icon themes don't ship .icon files anyway.

GtkIconTheme: Deprecate functionality related to .icon files

This functionality is not used at all in modern icon themes.

Drop the builtin icon cache

We don't need it anymore, now that the non-standard stock icons
are using resources.

Add non-standard stock icons as resources

This will allow us to remove the builtin icon cache in the future
while keeping the deprecated stock icons working.

Drop all standard icons from the builtin icontheme

We're going to require a complete icon theme, and we have
a test that checks for all the icons we use, so there is
no need to include all these fallback icons.

stylecontext: Deprecate regions

Regions are done in a very non-css way. They don't fit the DOM in that
they don't integrate into the CSS tree and they have very weird matching
behavior in selectors.

So I'm deprecating them now. GtkNotebook and GtkTreeview will continue
to use them and as long as they do, we can't remove the code for it.
But once those are ported it might be safe to remove the code as it will
clean up lots of places in the code by quite a bit.

stylecontext: Remove long outdated docs

The part about transition animations doesn't apply since 3.8.

gdkwindow-wayland: Destroy the wl_surface too when hiding a window

wl_surfaces can't switch roles, so destroying the xdg_surface but not
the wl_surface means that we could get an error when trying to re-map
the surface.

We could fix this by not destroying the xdg resource and only do it at
finalization time, but it's just as easy to just create a new wl_surface.

gdkwindow-wayland: Destroy the xdg roles before the wl_surface

Since the xdg roles are a special case of the surface, some compositors
like Weston destroy them automatically when the wl_surface is destroyed.
Thus, we need to destroy these first.

gdkwindow-wayland: Add a forgotten ref

gdkwindow-wayland: Another slight rearrange

gdkwindow-wayland: Pair a ref with its owner

gdkwindow-wayland: Don't post CONFIGURE events for the same size

The Wayland compositor is completely allowed to send us configure
events for the same size, and this validly happens if we're changing
states. Fizzle these out.

GtkTreeView: Remove a no-longer-valid optimization

The rubberband rendering code was assuming that we just have
a 1-pixel border and the rest of the rubberband is uniform.
That is not a safe assumption to make with css-styled
rubberbands, so remove it.

GtkIconView: Remove a no-longer-valid optimization

The rubberband rendering code was assuming that we just have
a 1-pixel border and the rest of the rubberband is uniform.
That is not a safe assumption to make with css-styled
rubberbands, so remove it.

Install the new icontheme test files

Adwaita: fix rubberband selection

Update Makefile.am for one more gitignore rule

Update git.mk

GtkIconTheme: Fix svg loading with FORCE_SIZE

Paradoxically, we were loading svgs at the nominal size when
FORCE_SIZE is specified, while scaling them exactly when it
isn't.
../build/win32/vs10/gtk.vcxproj

Add more icon theme tests

This test checks various directory size behaviours, with and
without the FORCE_SIZE flag.

Adwaita: hover state for horizontal spinbuttons

New window shadows

https://bugzilla.gnome.org/show_bug.cgi?id=731866

Adwaita: highligh not hilight...

Adwaita: remove wildcard for image effect

-gtk-image-effect: hilight now is set for .button:hover

Adwaita: calendar fixes

GtkHeaderBar: Don't forget to show the subtitle

https://bugzilla.gnome.org/show_bug.cgi?id=731790

Adwaita: flatten buttons slightly.

- gets rid of the specialcase for dark too.

Adwaita: more links styling

themes: use .popover class instead of GtkPopover

https://bugzilla.gnome.org/show_bug.cgi?id=731834

gtkpopover: set background and popover css classes

https://bugzilla.gnome.org/show_bug.cgi?id=731834

Adwaita: remove bg from insensitive backdrop spinbutton buttons

Adwaita: vertical spinbuttons inhert in backdrop state

Adwaita: actionbars' padding

Adwaita: reorderable tabs correctly styled for every notebook layout

Adwaita: set outline-color on selected items

Adwaita: checks and radios text focus ring fix.

Adwaita: minor fixes.

Adwaita: .scrollbars-junction

Updated Hebrew properties translation

Include image data in the builtin icon theme cache

When we switched the default for gtk-update-icon-cache to
not include image data, we should have thought about the
builtin icons. For them, we rely on image data being included.

This made the notify test fail in gnome-continuous, where we
build from git and always regenerate the builtin cache.